Skip to content

Conversation

@skateball
Copy link
Contributor

Updated links to OpenJDK project and added documentation references. Slim the README to the minimal required information and redirect for all other stuff to the https://sapmachine.io/docs as one single source of truth. That will ensure that we in future have only one single place to update information. Happy coding

update README

fixes #1029

Updated links to OpenJDK project and added documentation references.
Slim the README to the minimal required information and redirect for all other stuff to the https://sapmachine.io/docs as one single source of truth. That will ensure that we in future have only one single place to update information. Happy coding
@SapMachine
Copy link
Member

Hello @skateball, this pull request fulfills all formal requirements.

Copy link
Member

@RealCLanger RealCLanger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I also have a few updates in mind for the Readme but I'll do that in a separate PR.

@parttimenerd
Copy link
Contributor

parttimenerd commented Dec 12, 2025

Is it the intention not to list tools like the JFR events page or AP-Loader anywhere in the README (they are not currently listed in the docs either)?

Copy link
Contributor

@parttimenerd parttimenerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make sure the information isn't lost.

@RealCLanger
Copy link
Member

Is it the intention not to list tools like the JFR events page or AP-Loader anywhere in the README (they are not currently listed in the docs either)?

This information moved to https://sapmachine.io/docs/sap-in-openjdk. I think this should be linked more directly in the README.md. This will be part of the changes that I intend to do in a subsequent PR.

@RealCLanger RealCLanger merged commit 7185d8d into sapmachine Dec 12, 2025
106 checks passed
@RealCLanger RealCLanger deleted the update-readme.md branch December 12, 2025 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants